Feat: Support VCN Native CNI/Native Pod Networking changes #399

sindhusri16 · 2025-02-24T17:25:31Z

Feat: Support VCN Native CNI/Native Pod Networking changes

What this PR does / why we need it:
This includes new fields to support vcn native cni using native pod networking. Fields are introduced at the machine and machinetemplate level to include the cniType param. OKE on PCA needs this feature and could be supported when NPN is open as well.

Which issue(s) this PR fixes (optional, in fixes #(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #357

* Update to CAPI 1.7

…racle#367)

* Add support for kubernetes v1.29

* Add support for multiple CIDR in vcn and fix dns label bug

* Fix the label in CRD for versions and add more parameters to NLB spec

Using KUBERNETES_VERSION instead of OCI_MANAGED_KUBERNETES_VERSION will have a version mismatch and the managed nodes will fail to join.

…r both Loadbalancer and Machines (oracle#378)

Since we aren't really doing these anymore I'll remove the section.

…racle#384)

…#385) * changed wrClient to nlb and lb client to fetch core work-requests. * combine error events into single message * concatention changes

This was left out of the previous feature add.

…ment (oracle#395) * updated tilt-helper 1.updated tilt image golang version to 1.21.8 2.added bash to kustomize-sub.sh call Signed-off-by: Mayank Kakani <[email protected]> * Update types.go Added LaunchParavirtualizedVolumeAttachment types to v1 * Update types.go added LaunchParavirtualizedVolumeAttachment type * Update machine.go Added support for getParavirtualizedVolumeAttachment * Update infrastructure.cluster.x-k8s.io_ocimachines.yaml added launchParavirtualizedVolumeAttachment defination * Update infrastructure.cluster.x-k8s.io_ocimachinetemplates.yaml added launchParavirtualizedVolumeAttachment definations * Update go.mod Updated OCI go sdk version to v65.81.1 to support paravirtualized volume * Update go.sum Updated OCI SDK version * Update machine_test.go added test for paravirtualized volumes * Update machine.go updated message "Unknown attachment type not supported" * Update machine_test.go updated name - "check all params together, with paravirtualized volume support" * Fixed unittests Fixed paravirtualized volume support unit test case --------- Signed-off-by: Mayank Kakani <[email protected]>

v2 is deprecated https://github.blog/changelog/2024-02-13-deprecation-notice-v1-and-v2-of-the-artifact-actions/

…ges from open source

…luster-api-provider-oci into vcn-native-changes

sindhusri16 · 2025-02-28T19:21:50Z

Unittest output:
GOBIN=/home/opc/cluster-api-provider-oci/hack/tools/bin ./scripts/go_install.sh sigs.k8s.io/controller-runtime/tools/setup-envtest setup-envtest v0.0.0-20230131074648-f5014c077fc3
kube-builder assets: /home/opc/.local/share/kubebuilder-envtest/k8s/1.24.2-linux-amd64
KUBEBUILDER_ASSETS="/home/opc/.local/share/kubebuilder-envtest/k8s/1.24.2-linux-amd64" go test -coverprofile=coverage.out ./...
? github.com/oracle/cluster-api-provider-oci [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/config [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/metrics [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/scope/mocks [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/base [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/base/mock_base [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/compute [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/compute/mock_compute [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/computemanagement [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/computemanagement/mock_computemanagement [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/containerengine [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/containerengine/mock_containerengine [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/identity [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/identity/mock_identity [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/loadbalancer [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/loadbalancer/mock_lb [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/networkloadbalancer [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/networkloadbalancer/mock_nlb [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/vcn [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/vcn/mock_vcn [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/workrequests [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/workrequests/mock_workrequests [no test files]
? github.com/oracle/cluster-api-provider-oci/feature [no test files]
? github.com/oracle/cluster-api-provider-oci/version [no test files]
ok github.com/oracle/cluster-api-provider-oci/api/v1beta1 89.740s coverage: 23.6% of statements
ok github.com/oracle/cluster-api-provider-oci/api/v1beta2 0.052s coverage: 14.6% of statements
ok github.com/oracle/cluster-api-provider-oci/cloud/ociutil 0.012s coverage: 15.9% of statements
ok github.com/oracle/cluster-api-provider-oci/cloud/scope 20.148s coverage: 74.7% of statements
ok github.com/oracle/cluster-api-provider-oci/cloud/util 0.623s coverage: 61.1% of statements
ok github.com/oracle/cluster-api-provider-oci/controllers 2.077s coverage: 58.4% of statements
ok github.com/oracle/cluster-api-provider-oci/exp/api/v1beta1 31.177s coverage: 15.3% of statements
ok github.com/oracle/cluster-api-provider-oci/exp/api/v1beta2 0.028s coverage: 4.8% of statements
ok github.com/oracle/cluster-api-provider-oci/exp/controllers 0.374s coverage: 56.3% of statements
go tool cover -func=coverage.out -o coverage.txt
go tool cover -html=coverage.out -o coverage.html

joekr · 2025-03-03T17:17:54Z

e2e tests

Ran 7 of 28 Specs in 2637.402 seconds
SUCCESS! -- 7 Passed | 0 Failed | 0 Pending | 21 Skipped

joekr · 2025-03-03T20:18:30Z

cloud/util/util.go

+	npnVersion                      = "oci.oraclecloud.com/v1beta1"
+	npnKind                         = "NativePodNetwork"
+	apiExtensionVersion             = "apiextensions.k8s.io/v1"
+	npnCrdName                      = "nativepodnetworks.oci.oraclecloud.com"


Are these needed here now?

sindhusri16 · 2025-03-05T15:45:41Z

Unittests after the latest commit

make test
GOBIN=/home/opc/cluster-api-provider-oci/hack/tools/bin ./scripts/go_install.sh sigs.k8s.io/controller-runtime/tools/setup-envtest setup-envtest v0.0.0-20230131074648-f5014c077fc3
kube-builder assets: /home/opc/.local/share/kubebuilder-envtest/k8s/1.24.2-linux-amd64
KUBEBUILDER_ASSETS="/home/opc/.local/share/kubebuilder-envtest/k8s/1.24.2-linux-amd64" go test -coverprofile=coverage.out ./...
? github.com/oracle/cluster-api-provider-oci [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/config [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/metrics [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/scope/mocks [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/base [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/base/mock_base [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/compute [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/compute/mock_compute [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/computemanagement [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/computemanagement/mock_computemanagement [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/containerengine [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/containerengine/mock_containerengine [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/identity [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/identity/mock_identity [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/loadbalancer [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/loadbalancer/mock_lb [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/networkloadbalancer [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/networkloadbalancer/mock_nlb [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/vcn [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/vcn/mock_vcn [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/workrequests [no test files]
? github.com/oracle/cluster-api-provider-oci/cloud/services/workrequests/mock_workrequests [no test files]
? github.com/oracle/cluster-api-provider-oci/feature [no test files]
? github.com/oracle/cluster-api-provider-oci/version [no test files]
ok github.com/oracle/cluster-api-provider-oci/api/v1beta1 97.718s coverage: 23.6% of statements
ok github.com/oracle/cluster-api-provider-oci/api/v1beta2 0.053s coverage: 14.6% of statements
ok github.com/oracle/cluster-api-provider-oci/cloud/ociutil 0.005s coverage: 15.9% of statements
ok github.com/oracle/cluster-api-provider-oci/cloud/scope 20.314s coverage: 74.7% of statements
ok github.com/oracle/cluster-api-provider-oci/cloud/util 0.733s coverage: 61.1% of statements
ok github.com/oracle/cluster-api-provider-oci/controllers 2.161s coverage: 58.4% of statements
ok github.com/oracle/cluster-api-provider-oci/exp/api/v1beta1 31.113s coverage: 15.3% of statements
ok github.com/oracle/cluster-api-provider-oci/exp/api/v1beta2 0.059s coverage: 4.8% of statements
ok github.com/oracle/cluster-api-provider-oci/exp/controllers 0.943s coverage: 56.3% of statements
go tool cover -func=coverage.out -o coverage.txt
go tool cover -html=coverage.out -o coverage.html

shyamradhakrishnan and others added 25 commits April 27, 2024 10:01

Update to CAPI 1.7 (oracle#364)

9aa0b4e

* Update to CAPI 1.7

Fix docs for instance principal (oracle#365)

cf46124

Set instance in moving state to be not ready, but dont throw error (o…

b53ef08

…racle#367)

Update to k8s 1.29 (oracle#369)

5032281

* Add support for kubernetes v1.29

Add support for multiple CIDR in vcn and fix dns label bug (oracle#372)

9b214a5

* Add support for multiple CIDR in vcn and fix dns label bug

Fix webhook errors and add params to NLB (oracle#374)

d5d6800

* Fix the label in CRD for versions and add more parameters to NLB spec

Machinepool should use OCI_MANAGED_KUBERNETES_VERSION ENV (oracle#375)

ca7c47a

Using KUBERNETES_VERSION instead of OCI_MANAGED_KUBERNETES_VERSION will have a version mismatch and the managed nodes will fail to join.

feat: Add detailed work request errors when work-request is failed fo…

1b44dcd

…r both Loadbalancer and Machines (oracle#378)

doc: remove office hours from readme (oracle#381)

98636ee

Since we aren't really doing these anymore I'll remove the section.

Changed wrClient to nlb and lb client to fetch core work-requests. (o…

0a4c6e5

…racle#384)

Combine work-request errors into single reconcileError event. (oracle…

a5dc517

…#385) * changed wrClient to nlb and lb client to fetch core work-requests. * combine error events into single message * concatention changes

bug: Update the CRDs for the workrequest feature (oracle#387)

14936bf

This was left out of the previous feature add.

Update release workflow to use new upload-artifact@v4 (oracle#386)

5aa8d27

v2 is deprecated https://github.blog/changelog/2024-02-13-deprecation-notice-v1-and-v2-of-the-artifact-actions/

Increment metadata version (oracle#398)

e297ab6

Feat: API Spec & CRD & Conversion for VCN Native Params

fd4c2b6

Feat: NPN CR Reconciliation in OCIMachine_controller

a4f44a5

Fix: Add common label for compatibility & remove non-related crd chan…

f816852

…ges from open source

Fix: Improve NPN CR Reconcile

1d28aba

Fix: Kube Client Creation

b7af01c

Fix: Update CNI Type String

6e53cc9

Fix: Return nil upon retry

f090b7a

Fix: unused var

5875d6a

Fix: improve logging and error display

763cd93

Fix: Use different CR for get and create

10b419a

oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Feb 24, 2025

sindhusri16 and others added 4 commits February 24, 2025 22:55

Merge branch 'main' into vcn-native-changes

5e73d5e

Feat: include the NPN Reconcile into client scope package

b49e82a

Merge remote-tracking branch 'origin' into vcn-native-changes

885bdcf

Merge branch 'vcn-native-changes' of https://github.com/sindhusri16/c…

f737228

…luster-api-provider-oci into vcn-native-changes

sindhusri16 changed the title ~~Vcn native changes~~ Feat: Support VCN Native CNI/Native Pod Networking changes Feb 28, 2025

sindhusri16 requested a review from joekr February 28, 2025 19:12

add workrequestclientUrl to crd to remove inconsistency

8a7c9dc

joekr reviewed Mar 3, 2025

View reviewed changes

Style: Remove Redudant Const

5c653d3

dranicu and others added 2 commits March 19, 2025 09:56

Incrementing metadata version (oracle#401)

1da2921

Merge branch 'oracle:main' into vcn-native-changes

f12a2cb

sindhusri16 changed the base branch from main to release-v0.15.0 April 18, 2025 06:20

sindhusri16 closed this Apr 23, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feat: Support VCN Native CNI/Native Pod Networking changes #399

Feat: Support VCN Native CNI/Native Pod Networking changes #399

sindhusri16 commented Feb 24, 2025 •

edited

Loading

sindhusri16 commented Feb 28, 2025

joekr commented Mar 3, 2025

joekr Mar 3, 2025

sindhusri16 commented Mar 5, 2025

Feat: Support VCN Native CNI/Native Pod Networking changes #399

Feat: Support VCN Native CNI/Native Pod Networking changes #399

Conversation

sindhusri16 commented Feb 24, 2025 • edited Loading

sindhusri16 commented Feb 28, 2025

joekr commented Mar 3, 2025

e2e tests

joekr Mar 3, 2025

Choose a reason for hiding this comment

sindhusri16 commented Mar 5, 2025

sindhusri16 commented Feb 24, 2025 •

edited

Loading